Skip to content

fix(auto_labeling_3d): fix inconsistent deepen label fields#208

Open
jacoblambert wants to merge 1 commit into
mainfrom
fix/auto-labeling-3d_consistent-class-labels
Open

fix(auto_labeling_3d): fix inconsistent deepen label fields#208
jacoblambert wants to merge 1 commit into
mainfrom
fix/auto-labeling-3d_consistent-class-labels

Conversation

@jacoblambert
Copy link
Copy Markdown
Collaborator

Summary

Detection results and tracking results for class id may not match since some classes are tracked jointly (car, truck, bus). This causes issue in Deepen so we force the tracking class and detection to match, tracking class taking priority.

A more fundamental improvement of tracking to ensure bidirectional tracking/fixing of class labels based on entire track is left as future work, for now we only fix this at Deepen conversion.

Test performed

Run example_deepen.yaml on a few datasets and confirm the mismatch issue was fixed.

@jacoblambert jacoblambert requested a review from KSeangTan as a code owner May 8, 2026 06:38
@jacoblambert jacoblambert requested a review from tughril May 8, 2026 06:38
Copy link
Copy Markdown
Collaborator

@KSeangTan KSeangTan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall. Thanks for the fix!

derive the category from the assigned run-level ID instead of the frame label.
"""
unique_label_id = self.assign_id(uuid, label)
return unique_label_id.rsplit(":", maxsplit=1)[0]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docstring why it is [0]

Copy link
Copy Markdown

@tughril tughril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for fixing label_category_ids.
I'm ready to integrate this with webauto.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants